-
Notifications
You must be signed in to change notification settings - Fork 9
Update to 1.12.0 #137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Update to 1.12.0 #137
Conversation
|
@andrewdnolan, one thing we need to figure out is how to set permissions on the My thought here is that it might be safer to introduce a Could you look into this? The relevant code to fix would be: |
Changes default version of python to 3.13 and release versions to be 3.11, 3.12, and 3.13. To support the new python versions all CDAT related packages have been removed from the `meta.yaml` and the associated testing of `cdms2` in the `bootstrap.py` file. The `pcmdi_metrics` package is currently commented out of the `meta.yaml` because it has `genutils`, `cdutils`, `cdms2`, and `cdp` as dependencies. When a new version of `pcmdi_metrics`, without CDAT dependencies, is released it will be uncommented out of the `meta.yaml`. With the update to python 3.13 the build string for the python version has changed and is now dealt with accordingly.
A cmake based build system was introduce in version 2.3.0. The spack package recipe for TempestExtremes uses a cmake build system, so a version >=2.3.0 is now neeed
Also simplified the spack environment name given that the spack installation lives within a versioned e3sm-unified direcotry.
d9080aa to
64eb787
Compare
e3sm_supported_machines/bootstrap.py
Outdated
| base_path = config.get('e3sm_unified', 'base_path') | ||
| spack_base = f'{base_path}/{env_name}/{machine}/' | ||
| spack_base_path = ( | ||
| f'{base_path}/{env_name}/{machine}/spack/spack_for_{compiler}_{mpi}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this addition the directory layout(on chrysalis) would look like:
$ tree -L 4 /lcrc/soft/climate/e3sm-unified/
/lcrc/soft/climate/e3sm-unified/
├── e3sm_unified_x_x_x
│ └── chrysalis
│ ├── base
│ ├── spack
│ │ └── spack_for_gnu_openmpi
for e3sm-unified where there's only ever on compiler and mpi combination, maybe the intermidate spack folder is unneeded. Happy to defer to you about that @xylar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, the spack folder is redundant for now but consistent with future plans, right? Let's keep it for consistency.
recipes/e3sm-unified/meta.yaml
Outdated
| - jupyter | ||
| - livvkit 3.1.0 | ||
| - mache 1.28.0 | ||
| - mache 1.31.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
During deployment, please use the branch:
https://github.com/E3SM-Project/mache/tree/update-to-2.0.0
with the --mache_fork and --mache_branch flags.
Feel free to update that branch (which includes E3SM-Project/mache#303) as needed to get things working.
We can make a release candidate when we know that things are working but right now it feels premature.
e3sm_supported_machines/default.cfg
Outdated
| # the version of mache to use during deployment (should match the version used | ||
| # in the package itself) | ||
| mache = 1.28.0 | ||
| mache = 1.31.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can leave this for now but will eventually need to be 2.0.0rcX once we have a release candidate.
Update package version to be in sync with confluence page Co-authored-by: Xylar Asay-Davis <[email protected]>
Remove constraint on notebook which was for compatability with nb_conda, which has been removed. notebook has been left in, without constraint, so that jupyter notebooks are still avaialable
Group write permissionI enabled group write permission on the E3SM-Unified base directories on:
I don't have access to Ruby or Dane. I am renewing my Compy access. |
3c0c625 to
1f21059
Compare
This is necessary to avoid a SbangPathError in spack on Frontier. This merge also drops the requirement that the machine be specified. There is no longer an attempt to use a shared conda base environment as the conda environment for bootstrapping. Instead, the person deploying Unified must supply the `--conda` flag or have a base environment loaded before deploying.
The deploy and bootstrap scripts are now getting totally different conda bases in totally different ways so a shared function doesn't make sense anymore. This merge splits the functionality back into these 2 scripts for better clarity and simplicity.
Now consitent with the new layout for e3sm-unified
458a68e to
14d1df4
Compare
This is needed to unload a problem module on ALCF Polaris and may be useful on other machines in the future. This requires a new capability in mache.
b8c5960 to
ced1fb5
Compare
|
@xylar would mind looking over the most recent commit? Assuming you're OK with the approach/layout here, I'll get cranking on the NCO PR. |
|
@andrewdnolan, as discussed on slack, looks good! |
Co-authored-by: Xylar Asay-Davis <[email protected]>
Adds moab and mache RCs
2d2f1a1 to
577f929
Compare
In this version
Many packages have been updated, including:
python versions
3.11,3.12, and3.13will be supportedNew and improved
TBD
Deployment
Deployed on:
Sync Diagnostics
Synced on:
Delete old test envs